exports.writeConfig - Code Metrics - Inspection of "Create an extendible plugin framework #315 #316" - jmathai/elodie - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#317)

by Jaisen

created 2019-06-28 07:25 UTC

exports.writeConfig A

↳ Parent: app/modules/config.js

Complexity

Conditions	2
Paths	11

Size

Total Lines	18
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	2
eloc	14
nop	1
dl	0
loc	18
rs	9.7
c	1
b	0
f	1
nc	11

var exports = module.exports = {};
var fs = require('fs'),
    os = require('os'),
    defaultConfigFile = (function() {
      var f = __dirname;
      for(var i=0; i<2; i++) {
        f = f.substr(0, f.lastIndexOf(os.platform() == 'win32' ? '\\' : '/'));
      }
      return f + (os.platform() == 'win32' ? '\\config.ini-sample': '/config.ini-sample');
    })(),
    configFile = (process.env.HOME || process.env.USERPROFILE) + (os.platform() == 'win32' ? '\\.elodie\\config.ini' : '/.elodie/config.ini'),
    hasConfig,
    setConfig;


exports.hasConfig = function() {
  console.log(defaultConfigFile);

  console.log(configFile);
  return fs.existsSync(configFile);
};

exports.writeConfig = function(params) {
  var contents;
  try {
    if(exports.hasConfig()) {
      contents = fs.readFileSync(configFile).toString();
    } else {
      contents = fs.readFileSync(defaultConfigFile).toString();
    }

    console.log(contents);

    contents = contents.replace(/key=[\s\S]+$/,'key='+params['mapQuestKey']);
    fs.writeFileSync(configFile, contents);
    return true;
  } catch(e) {
    console.log(e);
    return false;
  }
};


1			var exports = module.exports = {};
2			var fs = require('fs'),
3			os = require('os'),
4			defaultConfigFile = (function() {
5			var f = __dirname;
6			for(var i=0; i<2; i++) {
7			f = f.substr(0, f.lastIndexOf(os.platform() == 'win32' ? '\\' : '/'));
8			}
9			return f + (os.platform() == 'win32' ? '\\config.ini-sample': '/config.ini-sample');
10			})(),
11			configFile = (process.env.HOME \|\| process.env.USERPROFILE) + (os.platform() == 'win32' ? '\\.elodie\\config.ini' : '/.elodie/config.ini'),
12			hasConfig,
13			setConfig;
			0 ignored issues – show Unused Code introduced 2019-01-24 14:20 UTC by Report Bug Copy Issue Report The variable `setConfig` seems to be never used. Consider removing it. Loading history...
14
15			exports.hasConfig = function() {
16			console.log(defaultConfigFile);
			0 ignored issues – show Debugging Code introduced 2019-01-24 14:20 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
17			console.log(configFile);
18			return fs.existsSync(configFile);
19			};
20
21			exports.writeConfig = function(params) {
22			var contents;
23			try {
24			if(exports.hasConfig()) {
25			contents = fs.readFileSync(configFile).toString();
26			} else {
27			contents = fs.readFileSync(defaultConfigFile).toString();
28			}
29
30			console.log(contents);
			0 ignored issues – show Debugging Code introduced 2019-01-24 14:20 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
31			contents = contents.replace(/key=[\s\S]+$/,'key='+params['mapQuestKey']);
32			fs.writeFileSync(configFile, contents);
33			return true;
34			} catch(e) {
35			console.log(e);
36			return false;
37			}
38			};
39

jmathai / elodie

Pull Request — master (#317)

exports.writeConfig A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like